Skip to content

Comments

wrapper for fastreseg#96

Open
dariarom94 wants to merge 3 commits intomainfrom
add_fastreseg
Open

wrapper for fastreseg#96
dariarom94 wants to merge 3 commits intomainfrom
add_fastreseg

Conversation

@dariarom94
Copy link
Contributor

Describe your changes

Added the implementation of fastreseg
Potential issues:

  • cell_type code and structure names in sdata are hardcoded
  • installation might need too long

par_input_scrnaseq='resources_test/task_ist_preprocessing/mouse_brain_combined/scrnaseq_reference.h5ad'
par_sc_cell_type_key='cell_type'
par_output='resources_test/task_ist_preprocessing/mouse_brain_combined/transcript_assignments.zarr'
par_transcripts_key='transcripts'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The par_transcripts_key and par_coordinate_system are not used below since keys were hardcoded in the scripts. It's fine to hardcode them, so either take out the two arguments from the config yaml and here (recommended way) or make the scripts support them as arguments (wouldn't do)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants